Fix a bug in Microsoft.Resources/deploymentScripts where tenantId property is missed from ManagedServiceIdentity.#11059
Fix a bug in Microsoft.Resources/deploymentScripts where tenantId property is missed from ManagedServiceIdentity.#11059markcowl merged 3 commits intoAzure:masterfrom lwang2016:master
Conversation
Mark environment variable secureValue as secret
Fix a bug in Microsoft.Resources/deploymentScripts where tenantId property is missed from ManagedServiceIdentity.
[Staging] Swagger Validation Report
️✔️ |
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-java - Release
|
azure-sdk-for-python-track2 - Release
|
azure-sdk-for-js - Release
|
azure-sdk-for-net - Release
|
azure-resource-manager-schemas - Release
|
azure-sdk-for-go - Release
|
Trenton Generation - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
Azure CLI Extension Generation - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
| "modelAsString": true | ||
| } | ||
| }, | ||
| "tenantId": { |
There was a problem hiding this comment.
Info for other reviewers: The service has always returned the tenant ID. @lwang2016 is fixing a swagger accuracy issue.
There was a problem hiding this comment.
Correct. Thanks for the clarification @majastrz
majastrz
left a comment
There was a problem hiding this comment.
Signed off from ARM since this is a swagger bug fix that updates swagger to match what the service has always returned (swagger accuracy flagged this)
This is a bug in deploymentScripts swagger spec while RP always returns the tenantId. The fix is to add the missing property.